Skip to content

Conversation

prestwich
Copy link
Member

@prestwich prestwich commented Sep 23, 2025

Add the flashbots provider, ported from the builder

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@prestwich prestwich force-pushed the prestwich/flashbots-provider branch from e971033 to 0e1ff7a Compare September 23, 2025 15:03
@prestwich prestwich force-pushed the prestwich/flashbots-provider branch from 0e1ff7a to e601de2 Compare September 23, 2025 15:03
@prestwich prestwich marked this pull request as ready for review September 23, 2025 15:05
@prestwich
Copy link
Member Author

i have run the integration test and verified that it passes


let value = self.compute_signature(&body_bz).await?;

let client = reqwest::Client::new();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we might want to move this into the struct itself to let people instantiate the Flashbots provider with a custom client, akin to how we're doing it on the tx cache client?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right now, i'm just porting. we can make improvements too if we want?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as you prefer :) i think we want to do this regardless, but i'm fine with a followup pr

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'll get it in rq, might as well while here

@prestwich prestwich force-pushed the prestwich/flashbots-provider branch from 09d6df8 to 25b7a62 Compare September 23, 2025 15:13

let value = self.compute_signature(&body_bz).await?;

let client = reqwest::Client::new();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as you prefer :) i think we want to do this regardless, but i'm fine with a followup pr

@prestwich prestwich force-pushed the prestwich/flashbots-provider branch from 25b7a62 to 2a7ee26 Compare September 23, 2025 15:21
@prestwich prestwich force-pushed the prestwich/flashbots-provider branch from 2a7ee26 to 8d390e7 Compare September 23, 2025 15:23
@prestwich prestwich enabled auto-merge (squash) September 23, 2025 15:24
@prestwich prestwich merged commit f22abd3 into main Sep 23, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants